Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Stack: Kyverno #38

Merged
merged 9 commits into from
Oct 2, 2024
Merged

Stack: Kyverno #38

merged 9 commits into from
Oct 2, 2024

Conversation

kurktchiev
Copy link
Contributor

This stack will install Kyverno for policy control and enforcement.

Optionally, a user can install the Kyverno PSS implementation for restricted in Audit mode. Furthermore, the user can instead enable the Enforce mode of PSS along with the necessary exceptions to allow for proper ref-implementation functioning.

Signed-off-by: Boris 'B' Kurktchiev <[email protected]>
… well as include the exceptions needed to make other Ref implementation components work out of the box

Signed-off-by: Boris 'B' Kurktchiev <[email protected]>
…dit or Enforce mode

Signed-off-by: Boris 'B' Kurktchiev <[email protected]>
Signed-off-by: Boris 'B' Kurktchiev <[email protected]>
Signed-off-by: Boris 'B' Kurktchiev <[email protected]>
@csantanapr
Copy link

This super cool @kurktchiev 🚀

name: crossplane-system-cnoe-operation
namespace: kyverno
spec:
exceptions:

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we install crossplane different to make it more align with best practice policies?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Am not sure what you mean here can you elaborate

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This adds exceptions to allow crossplane to work, is there some changes we can do the crossplane installation here to not need these many exceptions?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I could, but since Stacks have no conditionals, if the exceptions are installed without Kyverno, the installation will fail as k8s won't know what object that is. Let me know.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@nabuskey ^^ let me know if you have any thoughts

Copy link
Contributor

@nabuskey nabuskey Oct 2, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't see a path forward for this without native conditionals and parameter sharing in packages. Best we can do is to make changes to the Crossplane stack but it's outside scope of this PR. This kind of things is exactly why we need a way to support templating or layering somehow. I don't think we can expect people to wrap everything in a helm chart.

@csantanapr
Copy link

@kurktchiev It would be great to have you demo this in the next CNOE community meeting next week Oct 1st, https://docs.google.com/document/d/1Ir5EV8VFbXVW3O1N1X8cu8eTwcOZcPvrBqEdw1SY03I/edit

Signed-off-by: Boris 'B' Kurktchiev <[email protected]>
Copy link

@csantanapr csantanapr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@csantanapr csantanapr merged commit 2b12c4d into cnoe-io:main Oct 2, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants